Root route i.e. '/' shows Error 404. #3#5
Open
Jwright707 wants to merge 15 commits intoarnoob16:masterfrom
Open
Root route i.e. '/' shows Error 404. #3#5Jwright707 wants to merge 15 commits intoarnoob16:masterfrom
Jwright707 wants to merge 15 commits intoarnoob16:masterfrom
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/arnoob16/geeks-for-geeks-api/mn26df2xl |
Owner
|
Hey @Jwright707, Vercel Bot already deployed the changes you made Here's the screenshot of the error, try to fix this.
|
Author
|
Hi @arnoob16 , are you able to post the logs from vercel? I'm unable to view the logs and unable to replicate this locally. |
Owner
Hey @Jwright707, here are the logs from Vercel. |
Author
|
@arnoob16 Ignore that last commit, seems like its still not working. Have to keep digging into this issue. |
Author
|
@arnoob16 just a heads up. I'm unable to replicate this on my end, so there will be some commit pushed that you can ignore. Have to test some things out, but I'll keep you updated when a fix is there. Thank you for your patience and feedback. |
Moved the route back to app.py
|
@Jwright707 is attempting to deploy a commit to a Personal Account owned by @arnoob16 on Vercel. @arnoob16 first needs to authorize it. |
Author
|
@arnoob16 I believe the issue that I'm running into exists on the vercel.json file. I change the src for "/" to be redirected to the repo. |
arnoob16
reviewed
Jan 3, 2021
Owner
arnoob16
left a comment
arnoob16
left a comment
There was a problem hiding this comment.
You changes to vercel.json was authorized and deployed, now it doesn't show the earlier error page, but it doesn't solve the issue #3 either as the Error 404 is still shown at "/" route.
The deployment can be seen here ->
https://geeks-for-geeks-api-ar6037kjl.vercel.app/
Author
|
Hi @arnoob16 thank you. Yes, I believe I'm seeing whats happening. I'm going to take a walk and then look into this more. I believe I'm getting somewhere. Thank you again |
Author
|
Hi @arnoob16 I'm noticing that Vercel needs authorization now, I'm curious if this is due to me changing the vercel.json file. |
Author
|
@arnoob16 I believe this should be the last commit. I added back in the redirect because it seemed like the scrap.py was returning correctly. If this doesn't work then I may have to rethink this or I might need assistance. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added a redirect to the repository url when the URL is "/". This is connected to issue #3 .